I start wondering if rechunk might be a misleading function name. In the end, we are not rechunking the data but rather creating a view into the data that behaves as if it was chunked in a different way internally. There might be some confusion among users when they assume that the result of rechunk would magically make time series access more efficient on spatially chunked data, this happens quite reularly in the python world for xarray's chunk function.

I think we need either a different name or at least a better docstring explaining that no physical rechunking is happening here.

Yes, i am also wondering whether the name setchunks makes more sense. I could copy the docstring from the YAXArrays.setchunks function and remove the YAXArray specifics.

help?> YAXArrays.setchunks
  setchunks(c::YAXArray,chunks)

  Resets the chunks of a YAXArray and returns a new YAXArray. Note that this will not change the chunking of the underlying data itself, it will just make the data
  "look" like it had a different chunking. If you need a persistent on-disk representation of this chunking, use savecube on the resulting array. The chunks argument can
  take one of the following forms:

    •  a DiskArrays.GridChunks object

    •  a tuple specifying the chunk size along each dimension

    •  an AbstractDict or NamedTuple mapping one or more axis names to chunk sizes

Even setchunks seems more active than it is... It's really like fakechunks

Fakechunks sounds like it is not chunked at all. But I see your point.
Other possible names could be:

pretendchunk
pseudochunk
mockchunk

And if we rename the function we should also rename the main object because it would be good to have them in accordance.

Is there any conclusion/progress on this one. It would be a shame if this got blocked just because we can't agree on a name. So, maybe we discuss in our next big Raser call?

Can we use mockchunks and MockChunkedArray and get this merged?

I just did the renaming. As soon as this is in a mergeable state and someone had a look I am also going to rename the file from rechunk.jl to mockchunks.jl

@meggart if you're happy with the new name this looks good to merge to me

Oh one thing @felixcremer can we add const RechunkedDiskArray = MockChunksDiskArray or whatever it is so this isn't a breaking change?

Or maybe @deprecate

I will add a deprecation

I am happy with the new name. Feel free to merge